Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add division operator #38

Merged
merged 3 commits into from
Apr 13, 2017
Merged

Add division operator #38

merged 3 commits into from
Apr 13, 2017

Conversation

joemckie
Copy link
Contributor

As mentioned earlier today in #37, division is currently not supported and probably should be. This PR adds the / operator so that we can math properly 😉

fixes #37

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 98.962% when pulling 6b3f7c9 on joemckie:feature/support_division into 27c87e8 on vitalets:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.002%) to 99.308% when pulling c0d38d8 on joemckie:feature/support_division into 27c87e8 on vitalets:master.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.01%) to 99.317% when pulling 1311bc3 on joemckie:feature/support_division into 27c87e8 on vitalets:master.

@vitalets vitalets merged commit 633058c into vitalets:master Apr 13, 2017
@vitalets
Copy link
Owner

Thanks a lot for PR! 👍
Will release in 0.4.

@joemckie
Copy link
Contributor Author

Glad to help out! 😄

@joemckie joemckie deleted the feature/support_division branch April 13, 2017 12:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Variables don't support division
3 participants
@joemckie @coveralls @vitalets